Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: rollup boost plan errors #147

Merged
merged 2 commits into from
Feb 11, 2025
Merged

fix: rollup boost plan errors #147

merged 2 commits into from
Feb 11, 2025

Conversation

zhwrd
Copy link
Collaborator

@zhwrd zhwrd commented Jan 28, 2025

rollup boost nightly build is failing due to plan errors from merge / rebase issues

@zhwrd zhwrd enabled auto-merge (squash) February 10, 2025 18:18
@zhwrd zhwrd disabled auto-merge February 11, 2025 15:36
@zhwrd zhwrd force-pushed the zhwrd/fix-rollup-boost branch from 2305510 to d6440d5 Compare February 11, 2025 16:40
@zhwrd zhwrd merged commit 069b11d into main Feb 11, 2025
6 checks passed
@zhwrd zhwrd deleted the zhwrd/fix-rollup-boost branch February 11, 2025 16:49
zhwrd pushed a commit that referenced this pull request Feb 20, 2025
### Issue
Rollup-boost encountered issues due to configuration drift. 
### Description
While the primary drift problem was resolved in [PR
#147](#147),
follow-up testing uncovered additional inconsistencies that require
further adjustments.
This issue proposes targeted fixes based on local test results and
addresses the rest of inconsistencies.
Also couple of uncovered bugs were addressed (mainly with external
builder)
Removed
[this](https://github.com/ethpandaops/optimism-package/blob/0d490fd395ee579ea905bb4633e453e27e075bb8/src/el_cl_launcher.star#L354)
as it was causing a bug + was confusing
Changed behavior of builder consensus node. Now if external builder is
used it should setup consensus node outside of enclave too.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants